Skip to content

Refactor: file permissions #158

Merged
andrzej-janczak merged 2 commits intomainfrom
refactor/file-perm
Jun 30, 2025
Merged

Refactor: file permissions #158
andrzej-janczak merged 2 commits intomainfrom
refactor/file-perm

Conversation

@andrzej-janczak
Copy link
Copy Markdown
Contributor

refactor: Consolidate file permission constants and eliminate hardcoded values

  • Remove duplication between utils/files.go and constants/permissions.go
  • Delete utils/files.go and standardize on constants package for permissions
  • Update 14 files to use constants.DefaultFilePerms and constants.DefaultDirPerms
  • Replace hardcoded 0644 and 0755 values with named constants in tool runners
  • Improve code maintainability with single source of truth for file permissions

…ions

- Updated file and directory permission references throughout the codebase to use constants from the new constants package.
- Removed the utils package as it is no longer needed for permission management.
- Ensured consistent permission handling across various commands and configuration setups.
…nners

- Updated file and directory permission settings in tools (Dart Analyzer, Pylint, Lizard) to utilize constants from the new constants package.
- Ensured consistent permission handling across various tool runners, enhancing maintainability and readability of the codebase.
@codacy-production
Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% 21.74%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (66c17c5) 5539 1034 18.67%
Head commit (6958c6e) 5539 (+0) 1034 (+0) 18.67% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#158) 23 5 21.74%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@andrzej-janczak andrzej-janczak merged commit 8d3bece into main Jun 30, 2025
10 checks passed
@andrzej-janczak andrzej-janczak deleted the refactor/file-perm branch June 30, 2025 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants